Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add kubernetes_service option to teleport helm chart #5490

Merged
merged 12 commits into from
Feb 9, 2021

Conversation

stevenGravy
Copy link
Contributor

Allows for using kubernetes_service option instead of kubernetes enabled under proxy. This allows for pointing at multiple other k8s clusters with a kubeconfig.

Copy link
Contributor

@awly awly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stevenGravy could you please test this with:

  • proxy_service enabled, kubernetes_service not enabled
  • proxy_service not enabled, kubernetes_service enabled
  • both proxy_service and kubernetes_service enabled
  • both proxy_service and kubernetes_service enabled and proxy_service.kubernetes_public_addr set

examples/chart/teleport/templates/config.yaml Outdated Show resolved Hide resolved
examples/chart/teleport/values.yaml Outdated Show resolved Hide resolved
examples/chart/teleport/values.yaml Outdated Show resolved Hide resolved
stevenGravy and others added 4 commits February 8, 2021 14:34
Co-authored-by: Andrew Lytvynov <[email protected]>
Co-authored-by: Andrew Lytvynov <[email protected]>
@stevenGravy
Copy link
Contributor Author

stevenGravy commented Feb 8, 2021

@awly I confirmed all of the scenarios. Also reconfirmed with a optional kubeconfig_file
confirmed: proxy_service enabled, kubernetes_service not enabled
confirmed: proxy_service not enabled, kubernetes_service enabled
confirmed: both proxy_service and kubernetes_service enabled
confirmed: both proxy_service and kubernetes_service enabled and proxy_service.kubernetes_public_addr set

@stevenGravy stevenGravy merged commit 974825d into master Feb 9, 2021
@stevenGravy stevenGravy deleted the stevenGravy-patch/example/chart/teleport branch February 9, 2021 00:28
Joerger pushed a commit that referenced this pull request Feb 9, 2021
* Increment to version 0.0.10

* Add kubernetes_service option config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants